Skip to content

Conversation

@tmains515
Copy link
Contributor

@tmains515 tmains515 commented Nov 30, 2025

Summary by CodeRabbit

  • Documentation
    • Added a "Timeline for Completion" section to the README with an embedded image reference to enhance project documentation.

✏️ Tip: You can customize this high-level summary in your review settings.

tmains515 and others added 30 commits October 18, 2025 21:23
Signed-off-by: Tyler Mains <92330504+tmains515@users.noreply.github.com>
* EI-161 - Defined the Job table schema

* EI-161 | removing is_active

* EI-161 | Removed extra comma - syntax error

* Fixed INTEGRATION typo

---------

Co-authored-by: 2omb Finance <2ombfinance@protonmail.com>
* Chore: add Marcos to readme

* chore: Add Marcos to readme

* Revert "Chore: add Marcos to readme"

This reverts commit f9c78c9.

* EI-209 Created schedule table on database with attributes based on ERD.

* fixed minor ijmport errors and added JPA dependencies to pom.xml file

* Revert "fixed minor ijmport errors and added JPA dependencies to pom.xml file"

This reverts commit 10425d2.

* fixed minor import errors and added JPA dependencies to pom.xml file

* chore: add spring test profile disabling auto config for databases

* created as commented placeholders for classes that were not implemented yet.

* Added sql schedule table script, added lombook @Getter and @Setter support, added JPA @TeMPOraL notation.

---------

Signed-off-by: Marcos Pantoja <105100104+Mxrcos13@users.noreply.github.com>
Co-authored-by: Caleb Brock <cbrock@csus.edu>
Co-authored-by: Brynn Crowley <littlehill723@gmail.com>
* EI-137: Job Entity

Created Job Entity and added necessary dependency to pom.xml

* EI-137: Updates to Job Entity

Updated entity using lombok getter/setter. Put enums in separate file. Annotated with @TeMPOraL

* fix: clean up git merge remains

Signed-off-by: Brynn Crowley <littlehill723@gmail.com>

* feat: fix controller stub, add entity tests

Signed-off-by: Brynn Crowley <littlehill723@gmail.com>

* test: add more tests for job entity

Signed-off-by: Brynn Crowley <littlehill723@gmail.com>

* EI-137 Fixed Build Error

Build kept trying to connect to DB causing build fail. Added a test profile to exclude DataSource (similar to whats in main already)

---------

Signed-off-by: Brynn Crowley <littlehill723@gmail.com>
Co-authored-by: Brynn Crowley <littlehill723@gmail.com>
…nsights into feature/EI-26

# Conflicts:
#	endpoint-insights-api/pom.xml
#	endpoint-insights-api/src/main/resources/application.yaml
# Conflicts:
#	endpoint-insights-api/src/main/java/com/vsp/endpointinsightsapi/model/TestBatch.java
…nsights into feature/EI-25

# Conflicts:
#	endpoint-insights-api/src/main/java/com/vsp/endpointinsightsapi/config/EntityAuditConfig.java
#	endpoint-insights-api/src/main/java/com/vsp/endpointinsightsapi/controller/JobsController.java
#	endpoint-insights-api/src/main/java/com/vsp/endpointinsightsapi/model/AuditingEntity.java
#	endpoint-insights-api/src/main/java/com/vsp/endpointinsightsapi/model/Job.java
#	endpoint-insights-api/src/main/java/com/vsp/endpointinsightsapi/repository/TestBatchRepository.java
#	endpoint-insights-api/src/test/java/com/vsp/endpointinsightsapi/model/JobEntityTest.java
@coderabbitai
Copy link

coderabbitai bot commented Nov 30, 2025

Walkthrough

README.md was updated to add a "Timeline for Completion" section with an embedded image reference, placed after the Contributing section. This is a documentation-only change with no code or functional modifications.

Changes

Cohort / File(s) Summary
Documentation Update
README.md
Added "Timeline for Completion" section with image reference for project timeline documentation

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify image path/reference is correctly formatted and accessible
  • Confirm section placement and markdown formatting consistency with existing documentation

Possibly related PRs

  • Project timeline #62: Adds a parallel "Timeline for Completion" section with timeline image to docs/index.md, suggesting coordinated documentation updates across multiple files.

Poem

🐰 A timeline emerges, bold and clear,
Milestones mapped for all to peer,
In README's fold, the path is bright,
Guiding travelers through the night! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Project timeline' directly relates to the main change, which adds a new 'Timeline for Completion' section to the README documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch project-timeline

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Nov 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (1)
README.md (1)

121-121: Improve the image alt text for accessibility.

The alt text "Image" is generic and not descriptive. This impacts accessibility (WCAG compliance) and SEO. Consider updating it to something more descriptive like "Project Timeline for Completion" or similar.

Apply this diff to improve the alt text:

-![Image](./docs/document-assets/timeline.png)
+![Project Timeline for Completion](./docs/document-assets/timeline.png)
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5fdd98b and 3bb22a1.

⛔ Files ignored due to path filters (2)
  • endpoint-insights-ui/package-lock.json is excluded by !**/package-lock.json
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • README.md (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Jenkins / Linting and Unit Tests

@tmains515 tmains515 merged commit af5c118 into develop Dec 1, 2025
6 checks passed
@tmains515 tmains515 deleted the project-timeline branch December 1, 2025 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants